From: Steve Frécinaux <code@istique.net>
Date: Thu, 15 Oct 2009 22:34:23 +0000 (+0200)
Subject: [AppSwitcher] Reimplement the separator using St.Bin
X-Git-Tag: archive/raspbian/50.5-1+rpi1~1^2~1013^2^2~10720
X-Git-Url: https://dgit.raspbian.org/%22http://www.example.com/cgi/%22/%22http:/www.example.com/cgi/%22?a=commitdiff_plain;h=8e9549c6f176d1b21df58bf83d85063e91b7d21d;p=gnome-shell.git

[AppSwitcher] Reimplement the separator using St.Bin

This way it can be styled using CSS.
This can help to fix bug 597362.

https://bugzilla.gnome.org/show_bug.cgi?id=598651
---

diff --git a/data/theme/gnome-shell.css b/data/theme/gnome-shell.css
index 42097fe8d..dfabaacae 100644
--- a/data/theme/gnome-shell.css
+++ b/data/theme/gnome-shell.css
@@ -174,3 +174,8 @@ StScrollBar StButton#vhandle:hover
     border-radius: 4px;
     background: rgba(255,255,255,0.33);
 }
+
+.switcher-list .separator {
+    width: 1px;
+    background: rgba(255,255,255,0.33);
+}
diff --git a/js/ui/altTab.js b/js/ui/altTab.js
index 62b95e327..570b5943e 100644
--- a/js/ui/altTab.js
+++ b/js/ui/altTab.js
@@ -18,8 +18,6 @@ const POPUP_ARROW_COLOR = new Clutter.Color();
 POPUP_ARROW_COLOR.from_pixel(0xffffffff);
 const TRANSPARENT_COLOR = new Clutter.Color();
 TRANSPARENT_COLOR.from_pixel(0x00000000);
-const POPUP_SEPARATOR_COLOR = new Clutter.Color();
-POPUP_SEPARATOR_COLOR.from_pixel(0x80808066);
 
 const POPUP_APPICON_SIZE = 96;
 const POPUP_LIST_SPACING = 8;
@@ -385,11 +383,7 @@ SwitcherList.prototype = {
     },
 
     addSeparator: function () {
-        // FIXME: make this work with StWidgets and CSS
-        let box = new Big.Box({ padding_top: 2, padding_bottom: 2 });
-        box.append(new Clutter.Rectangle({ width: 1,
-                                           color: POPUP_SEPARATOR_COLOR }),
-                   Big.BoxPackFlags.EXPAND);
+        let box = new St.Bin({ style_class: 'separator' })
         this._separator = box;
         this._list.add_actor(box);
     },